broadway: Add missing dependency to build

The generated file clienthtml.h is #included by broadway-server.c, which
is one of the sources of the broadway library — so clienthtml.h needs to
be one of the sources of that library too.

Signed-off-by: Philip Withnall <withnall@endlessm.com>
https://bugzilla.gnome.org/show_bug.cgi?id=790489

broadway: Fix toplevel handling

The toplevels list contains GdkWindowImplBroadways, and we shouldn't
free it.

gdk: Remove gdk_flush()

If you need to flush a display, use gdk_display_flush().

There is never a need to flush displays you don't know about.

gtk-demo: No need to gdk_flush() here

And if there was a need, GDK should have done it.

toolbar: Remove gdk_flush() calls

It's not 2003 anymore.

main: Don't gdk_flush() after every main loop wait

For a start, this makes gtk_main() work different from g_main_loop_run()
calls.

But most importantly, modern GDK does proper syncing itself and doesn't
need to rely on a catch-all to get it right.

Add a comment

Document that gdk-private.h is meant for private gdk<>gtk apis.

Remove gdkprivate.h

This is one-too-many private headers. Move the few
declarations here to gdkinternals.h.

Drop generic error trap api from the docs

Drop gdkmain.h

Stop including gdkmain.h

It is empty now.

Drop an empty file

Nothing deprecated left!

Drop the error trap vfuncs

No longer used.

Drop the generic error trap api

This is only implemented on X11, so we don't
need generic api for it.

tests: Test the x11-specific error trap api

The other one is gone.

x11: Fix error traps in the dnd code

The code for grabbing keys was using a generic
error trap call, and more concerning, it was forgetting
to pop the trap.

selection: Use x11-specific traps

The generic trap api is going away.

mountoperation-x11: Use x11-specific traps

The generic trap api is going away.

Drop an unused variable

These are no longer used.

Drop a duplicate definition

GDK_PRIORITY_EVENTS is already defined in gdkevents.h

Drop gdk_set_program_class

Together with the corresponding getter.
This is x11-specific functionality.

x11: Add gdk_x11_display_set_program_class

The program class is a very X-specific concept.
There should be x11-backend api to deal with it.

wayland: Stop using gdk_get_program_class

This is an abuse, at best. If we need to pass the
app id down here, lets pass the app id.

Remove gdk/gdkx.h

It was just a kludge.

Move some code around

Move gdk_disable_multidevice to gdkdisplaymanager.[hc].

Drop gdk_notify_startup_complete

This function and its variant are just wrappers
around the display-specific function.

gtk: Stop using gdk_notify_startup_complete_with_id

This function is about to go away. Instead, use the
display-specific function that this is a wrapper of.

gdk: Move some headers around

Move gdk_flush and gdk_set_allowed_backends to where they
belong.

Drop gdk_beep

This is an unnecessary wrapper around gdk_display_beep.

im context thai: Stop using gdk_beep

It is about to go away.

Drop gdk_set_double_click_time

This is an unnecessary and not-recommended-to-be-used
wrapper around gdk_display_set_double_click_time.

selection: Remove the info uint

Instead of allowing people to pass a uint user-data, insist on them
comparing mime types.

The user data was a uint instead of a pointer anyway, so uniqueness
could not be guaranteed and it caused more issues than it was worth.
And that's ignoring the fact that it basically wasn't used.

selection: Remove gtk_target_table_*() functions

textview: Stop playing shenanigans with the target list

This breaks GtkSourceView adding custom targets on top of GtkTextView,
but we'll fix that later.

selection: Make GtkTargetEntry not allocatable

And remove it being registered as a boxed type (lol).

HighContrast: misc fixes

The HighContrast theme was not parsing anymore, due to
leftover widget style properties, and some missed cleanups,
like -gtk-icon-effect. Also update for the new focus handling,
and make checks and radios sharp again.

Adwaita: Make checks and radios sharp again

The assets we have are 14x14, set -gtk-icon-size to reflect that.

Change icon-size properties

We no longer support registering custom icon sizes, so
we can make these properties just enums. This also lets
us specify them by nick in ui files. Nice!

image: Update docs

Mention the new icon-size style classes, update the docs
for removed arguments and return values, and add new
functions to the docs.

Document -gtk-icon-size

Add this property to the list of CSS properties.

Support -gtk-icon-size for builtins

This makes css icons more similar to themed icons, which
is the overall goal here.

Drop gtk_icon_size_lookup

Add a new, private gtk_image_get_image_size to replace it, and
update the remaining callers in a suitable way.

recentchooser: Use GIcon instead of GdkPixbuf

iconhelper: Query size via CSS

Instead of looking at the icon size, look at the CSS value for
-gtk-icon-size. Set style classes depending on icon size instead.

Trivially change Adwaita and HighContrast to report the same values as
before.

enums: Change GtkIconSize values

The new values are the ones we intend to keep. The old ones had
duplicated meanings and nobody knew which one to take.

image: Don't take icon size in set_from_definition()

filechooserwidget: icon size is always 16

filechooserbutton: icon size is always 16

So define it that way.

cellrendererspinner: Don't track old icon size

It was only used to avoid gtk_icon_size_lookup() calls and those aren't
expensive.

image: Remove icon-size argument from icon setters

Instead, add a function gtk_image_set_icon_size() for the cases where
overriding the icon size is necessary.

Treat icon sizes the same way as pixel sizes, too. So gtk_image_clear()
no longer unsets the icon size.

image: Change getters

Instead of returning the icon size with them, make
gtk_image_get_icon_name() and gtk_image_get_gicon() only return the icon
itself.

As a benefit, we can turn them into regular getters that return values
instead of requiring out parameters.

Instead, provide gtk_image_get_icon_size() to query the icon size.

button: Get rid of icon size

In fact, make gtk_button_new_from_icon_name() just set the icon-name
property instead of creating a GtkImage.

scalebutton: Remove icon size

toolbar: Remove icon sizes

tooltips: Remove GtkIconSize usage

css: Add -gtk-icon-size

This will replace GtkIconSize in future patches.

introspection: Fix build

wayland: Fix initial cursor

Make sure the initial cursor isn't random which would happen due to an
early exit when cursor == pointer->cursor triggered because both were
NULL.

gdk: Replace GDK_NONE with NULL

gdk: Turn GdkAtom into a const char *

Instead of an integer, it is now a char pointer. We also use
g_intern_string() instead of doing the interning ourselves.

gdk: Make GDK_NONE Atom translate to NULL

Instead of the string "NONE" we now use NULL.

gdk: Get rid of magic special atoms

x11: Remove special casing for GdkAtoms

This is in preparation for the GdkAtom removal

dnd: Make GtkDragDest and GtkDragSource use GtkTargetList

This gets rid of GtkTargetEntry in the API and consistently uses
GtkTargetList.

clipboard: Consistently use GtkTargetList

Some code was using GtkTargetList, some used GtkTargetEntry and some
GtkTargetPair.

selection: Remove #ifdef WAYLAND

Instead, turn the functions into backend API:
  gdk_broadway_display_add_selection_targets()
  gdk_broadway_display_clear_selection_targets()
Remove the old per-backend functions, too.

cellrendererpixbuf: Remove stock-detail property

GTK3 didn't use that property already...

messagedialog: Remove unused header

win32: Remove all checks for GDK_WINDOW_ROOT

We are no longer creating windows of this type, so no
need to check for it.

win32: Remove all uses of get_root_window

This is following similar changes done in the x11 and
broadway backends, but it has not been built.
Some fixups may be needed.

placessiderbar: Fix sunny compilation

Make compilation not warn about unused variables when compiling with
HAVE_CLOUDPROVIDERS undefined.

widget: No need to update events

GtkWindow queues for all events anyway, so the needed ones are already
there.

gdkwindow: Remove event_mask arguments from constructors

window: Set ALL_EVENT_MASK

This is a trivial commit that does a big change: We now ignore event masks.

Further commits will clean up code, but if bisection ends up here, you
know it's because code is getting delivered events that it weren't getting
before.

gdk: Fix GDK_ALL_EVENTS_MASK

This mask was forgotten to update when the last 2 event masks were
added, probably because it looks like it's already maxed.

widget: Remove gtk_widget_set_device_enabled()

Nobody is using it.

x11: Remove mentions of GDK_WINDOW_ROOT

This backend never creates a window of this type, so no
need to check for it anymore.

x11: Stop creating a root window

We no longer need it.

x11: Store toplevel list in GdkDisplay

This will let us get rid of the root window.

x11: Remove another use of the root window

We just need the X root window here.

icontheme: Add a trailing / when enumerating resources

This avoids extra string copies in GResource.

gtkplacessidebar: Fix new tab/window handling for cloud accounts

It wasn't taking into account whether the sidebar had support for them
or not, resulting in a file chooser with open in new tab/window menu
items when it's not supported.

To fix it, do as with the other menus and check for the availability of
new tab/window flags.

https://bugzilla.gnome.org/show_bug.cgi?id=786123

gtkplacessidebar: Adapt to libcloudproviders 0.2.x

And a few improvements on the way.

https://bugzilla.gnome.org/show_bug.cgi?id=786123

application: Append a / to the icon resource path

g_resources_enumerate_children expects the path to end
in a '/' (even though thats not stated in the docs), and
will copy it if that isn't the case. Avoid the copy
by putting  a '/' there to begin with.

icontheme: Append a / to the resource path

g_resources_enumerate_children expects the path to end
in a '/' (even though thats not stated in the docs), and
will copy it if that isn't the case. Avoid the copy
by putting  a '/' there to begin with.

a11y: Handle a treeview with no columns

Code was spewing criticals to stderr because of nonexisting columns. So
check that there's actually an existing column first.

treeview: Only update vadjustment in size_allocate()

The 2 calls even do different things, but because the one in allocate
always overrides the one here...

treeview: Pass height to allocate as argument

Instead of exporting a function to query it.

progresstracker: Don't hand out NaN

When the duration is set to 0, clamp it to 1us. This way we're almost
correct: We should really instantly finish, but we don't. But we do
respect the delay.

Doing this properly would require some refactoring of how the progress
tracker actually maintains progress, and this is just a quick fix.

treeview: Use gtk_widget_get_width()

... instead of gtk_widget_get_allocated_width() and same for height.

This makes the treeview do the right thing when used with padding.

treeview: Don't allocate columns outside of size_allocate()

treeview: Fix RTL column header allocation

treeview: width_changed is always true, so remove it

Also, sanitize the RTL correction code that made sure resizing the width
of a treeview would keep the contents glued to the right border instead
of the left border.

x11: Fix a crash

This was crashing the moment a second window is opened.

Fix up managed dnd

Since commit 6c56d04cee23d3340cf7cf08756c610efe9ccdab,
we were doing some calls twice in the managed dnd case,
leading to double drops.

textview: Remove GDK windows

textview: Simplify gtk_text_view_buffer_to_window_coords()

... and gtk_text_view_window_to_buffer_coords()

textview: Use text_window_get_width()/height()

Instead of poking the GdkWindow. They have the same size after all.

textview: Make the semi-private headrs really private

Rename the files to have the private.h ending.

And remove gtktextdisplay.h from the installed files.

textview: Store border window sizes in a custom struct

That way, we don't need to have a GtKRequisition in every TextWindow.